Skip to content

feat: Add task dependency validation and visualization#1

Closed
nevil06 wants to merge 1 commit intomithun50:masterfrom
nevil06:master
Closed

feat: Add task dependency validation and visualization#1
nevil06 wants to merge 1 commit intomithun50:masterfrom
nevil06:master

Conversation

@nevil06
Copy link
Copy Markdown

@nevil06 nevil06 commented Dec 27, 2025

  • Added validateTaskDependencies() to detect circular dependencies
  • Added getTaskGraph() for visual dependency representation
  • New MCP tools: validate_task_dependencies and get_task_graph
  • Fixed lint warnings (removed unused variables)
  • Updated README with dependency management section
  • All tests passing, build successful

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • CI/CD changes
  • Other (please describe):

Related Issues

Fixes #

Changes Made

Testing

Test Configuration

  • Node.js version:
  • Operating System:
  • AI Client tested with:

Tests Performed

  • Unit tests pass (npm test)
  • Build succeeds (npm run build)
  • Lint passes (npm run lint)
  • Manual testing with MCP client
  • Manual testing with HTTP API

Test Cases

Screenshots / Logs

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally
  • Any dependent changes have been merged and published

Breaking Changes

Additional Notes

- Added validateTaskDependencies() to detect circular dependencies
- Added getTaskGraph() for visual dependency representation
- New MCP tools: validate_task_dependencies and get_task_graph
- Fixed lint warnings (removed unused variables)
- Updated README with dependency management section
- All tests passing, build successful
@mithun50
Copy link
Copy Markdown
Owner

@nevil06 please fix the failed issues also ignore the package-lock.json file.

@mithun50 mithun50 closed this Dec 27, 2025
@mithun50 mithun50 reopened this Dec 27, 2025
@mithun50 mithun50 self-requested a review as a code owner December 27, 2025 17:44
@mithun50 mithun50 closed this Dec 27, 2025
@mithun50
Copy link
Copy Markdown
Owner

@nevil06 fetch the latest commit and start working on it your old implimentation is broken.

Copy link
Copy Markdown
Owner

@mithun50 mithun50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Broken

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 1, 2026

This pull request has been automatically locked since there has been no activity for 30 days after it was closed.

If you have related changes, please open a new pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants